Skip to content

Conversation

@tacerus
Copy link
Member

@tacerus tacerus commented Jun 24, 2023

This is intended to become a replacement for the messy LUN management code in the orchestra formula. The functionality is now implemented with proper Salt modules and correct test=True operation, and is easily expandable for ONTAP operations beyond LUN management. Ironically, this replacement still uses Ansible as a middleware, but the new layout should make switching to direct interaction with the NetApp Python library easier should this be desired in the future.

tacerus added 11 commits June 22, 2023 03:31
New formula for managing NetApp ONTAP devices. This is intended to
replace the LUN/disk management parts in the messy orchestration
modules.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Handle executions with multiple tasks. Facilitate more REST operations
in results parsing.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Split into two functions, one to query whether the LUN is mapped at all,
and onne to query details about the mapping.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Initial state module.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
@tacerus tacerus force-pushed the new/netapp_ontap branch from fff4e7e to 0982673 Compare June 24, 2023 06:56
tacerus added 2 commits June 24, 2023 08:58
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
@tacerus tacerus force-pushed the new/netapp_ontap branch from 0982673 to bc7b084 Compare June 24, 2023 06:58
tacerus added 15 commits June 25, 2023 22:48
Add some comments, make igroup filter in get_lun_mapped() optional, show
comment in get_lun() output, drop single-use _config() function.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Add basic explanations to the functiosn in ontap_native.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Split into get_lun_mapping() and get_lun_mappings().

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Drop useless logic for multiple mapping results.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Add unmap_lun() for unmapping of a single LUN and the
matching lun_unmapped() state function.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
The Ansible based "ontap" execution module is not pursued anymore.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
These are no longer used, as the Salt modules now communicate with the
ONTAP Python library natively.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Drop old mapping logic (LUN's are now to be mapped using "lun_mapped"
instead), use paths instead of comments for validation.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Initial test structure.

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
- use shared LUN fixture
- add coverage for deletion and update operations

Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
tacerus added 6 commits June 29, 2023 20:53
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
Signed-off-by: Georg Pfuetzenreuter <georg.pfuetzenreuter@suse.com>
@tacerus tacerus force-pushed the new/netapp_ontap branch from 5eea98a to b908884 Compare July 2, 2023 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant